Massive performance improvement with large [25k+] documents in WPF.#95
Massive performance improvement with large [25k+] documents in WPF.#95ben-abraham wants to merge 2 commits intommanela:masterfrom
Conversation
…easily handle 10-20k+ line files. - Fully convert InternalLinesViewer to using virtualized component rendering - Added a basic ViewModel for InternalLinesViewer (LineViewerLineData). Supports dynamic visibility changes.
Cleaned up the GoTo helper functions significantly.
|
Note: a big missing feature with this now, is that you can't highlight/select the text at all since they are now TextBlock objects. There are a few solutions that could fix it I think, but they would be fairly involved. |
|
@ben-abraham Thanks for contributing. It's an interesting change although making text not selectable is a bummer. @kingcean can you take a look and share any thoughts? |
|
Looks good! |
|
Sorry for the lack of update, I'm trying to implement this with Document/Span elements directly without virtualization, which seems performant enough by itself, and should allow selection to work as well. Just having trouble getting the look to match quite right, so I kind of have to rework the WPF styles to make it work correctly. Also a few bugs related to scroll bars that need to be sorted out. |
|
@ben-abraham It has been a while, so pinging on this. Are you still interested in moving this forward (with or without your follow up changes). I am looking at doing some housekeeping and seeing if this is something we can get it to help it be lean and fast. |
I think this is the issue I was seeing with #143 |
4 participants
Can now easily handle 25k+ line files in side-by-side mode.
You can increase the repetitions in
MainWindow.xaml.csto 1000 to get a 25k line document, which is handled on my system easily.Let me know if there are any improvements you can offer. The style mechanism could be improved to be more pure-wpf, and I think the font properties might already inherit by default (although I'm not 100% sure about this one, I need to test it)